Skip to content

Update adapter tutorial with new registry instructions#22

Open
crystalxyz wants to merge 2 commits intoharbor-framework:mainfrom
crystalxyz:update-registry-format
Open

Update adapter tutorial with new registry instructions#22
crystalxyz wants to merge 2 commits intoharbor-framework:mainfrom
crystalxyz:update-registry-format

Conversation

@crystalxyz
Copy link

To reflect the breaking changes introduced by harbor-framework/harbor#1285, we are updating the adapter tutorial to include new dataset registry instructions.

Comment on lines +37 to +39
<Callout title="CLI command changes">
Harbor CLI commands have moved from plural to singular form (e.g., `harbor datasets` → `harbor dataset`, `harbor adapters` → `harbor adapter`, `harbor jobs` → `harbor job`, `harbor trials` → `harbor trial`). The old plural forms are still supported for backwards compatibility.
</Callout>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need this callout bc we made all the commands backwards compatible in case ppl have muscle memory or stale docs

```bash
# Run on entire local dataset
uv run harbor jobs start -p datasets/<your-adapter-name> -a <agent-name> -m <model-name>
harbor job start -p datasets/<your-adapter-name> -a <agent-name> -m <model-name>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

harbor job start can just be harbor run


# Edit dataset.toml with descriptions, authors, credits
# Then submit to Harbor team for publishing
harbor auth login
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably best not to include these steps bc they may get confused and publish without us?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants